-
Notifications
You must be signed in to change notification settings - Fork 365
Bug 1923923 - Allow creation or update (annotation) of an internal issue #8532
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
This is still a Work In Progress, the below screenshot shows the design I had in mind. |
In the current design, the user has to focus on two things:
|
adafa57
to
ea5686b
Compare
ea5686b
to
249a4f0
Compare
I'm actually stuck by an obscure JS error (that seems related to webpack build & react) that happens while submitting the form (when user is logged on treeherder):
@Archaeopteryx does that mean anything to you? |
Could this be the callback which currently adds the bug number returned by the bug filer to the pinboard? |
249a4f0
to
aa737b2
Compare
I recorded a video using the UI to create and reference an internal issue: https://github.com/user-attachments/assets/ce94f6b0-b2a8-4fba-a5d9-fc24e27cdd13.
Next (and last for this feature) work is to link a bugzilla ID to the internal issue once it gets creates, then update its attributes when Bugscache is automatically filled. |
Thank you for the video and the description. Does a third classification with a bug automatically reopen the bug filer? |
Actually it's the fourth classification that opens the bug filer (which is shown at the end of the video). About step 6, indeed it reuses the existing internal issue, then the UI also create a classification with the selected job. |
0c877ce
to
e0dfeb0
Compare
I rebased this branch, it is fine by me. I think I reproduced the bug I mentioned on thursday. It only happens with some special summary (e.g. |
Getting back to this tomorrow. |
Thank you. The changes have been deployed to prototype. |
I played a bit on prototype: https://prototype.treeherder.nonprod.cloudops.mozgcp.net/jobs?repo=autoland&revision=cc5967ffe3435273d130a6b4e992606e1948fe81&selectedTaskRun=aX1fdZkFQyuwljxSLA6e4Q.0 |
Remaining issues from testing the stack and PR 8595:
|
const existingBug = internalBugs.filter( | ||
(bug) => bug.internal_id === data.internal_id, | ||
)[0]; | ||
// Check if we reached the required number of occurrence to open a bug in Bugzilla |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this be used when a new classification with an internal issue gets created with pinning (instead of 'creating' an internal issue again like here)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is much harder to implement because we are not sure the bug suggestions are available (we are out of the scope of the FailureSummaryTab
). We could however support the case where the bug is part of the actual suggestions (thus reusing the above code). This is probably doable via a custom event.
If I understand correctly, cache for suggestions happens at job level in treeherder: https://github.com/mozilla/treeherder/blob/master/treeherder/model/error_summary.py#L125. I see no summary based cache for the suggestions query. To my understanding already opened jobs should not show the internal issue, but a new job with a TextLogError matching the summary should be visible. I'm implementing the 3 other points. |
I implemented the suggestions, except for the last point (number of occurrences is reached via pinning). I'll work on it next week. I had trouble to log in from my local application (making UI devs more complex to test…). It was working before, now I have the above error at https://auth.mozilla.auth0.com: |
No description provided.